fix(web): work log Show more — cursor, transition & scroll#1569
fix(web): work log Show more — cursor, transition & scroll#1569Marve10s wants to merge 4 commits intopingdotgg:mainfrom
Conversation
The raw <button> element was missing cursor-pointer, causing the default arrow cursor on hover instead of the expected hand pointer.
Use the existing Base UI Collapsible component to animate the overflow entries in tool call / work log groups with a 200ms height transition instead of an instant show/hide.
github-actions
bot
added
size:M
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Marve10s
force-pushed
the
fix/show-more-cursor-and-transition
branch
from
59c3df1 to
8e290fc
Compare
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Expanding a work log group during a running session would trigger scheduleStickToBottom via the timelineEntries effect, yanking the viewport to the bottom. Disable auto-scroll on toggle so the user stays where they clicked; the scroll-to-bottom button is shown if they want to jump back.
Marve10s
force-pushed
the
fix/show-more-cursor-and-transition
branch
from
954a90d to
8caccfb
Compare
2026-03-30.12.07.23.mp4Sorry, took a second for Codex and I also needed to approve access to folders for Claude :) |
1 participant
Summary
cursor-pointer, showing a default arrow on hover instead of a hand. Fixed by switching to the existingCollapsibleTriggercomponent which includes it.Collapsible+CollapsiblePanelcomponent, giving a smooth 200ms height animation when expanding/collapsing overflow entries.scheduleStickToBottom. Now disables auto-scroll on toggle so the user stays where they clicked; the scroll-to-bottom button appears if they want to jump back.Note
Low Risk
Low risk UI/UX change limited to work-log expand/collapse behavior and chat auto-scroll flags; main risk is subtle scroll state regressions during active sessions.
Overview
Work log/tool-call groups now expand/collapse via
Collapsibleinstead of a custom button, giving consistent pointer cursor styling and a smooth height transition.Toggling a group now keeps overflow entries mounted (
keepMounted) and changes the visible/hidden slicing so the newest work entries remain visible while older entries live in the collapsible panel.Auto-scroll is suppressed when expanding/collapsing a group:
ChatViewcancels any pending stick-to-bottom scroll, disables auto-scroll, and forces the scroll-to-bottom pill to appear so the viewport doesn’t jump.Written by Cursor Bugbot for commit 66cb7cf. This will update automatically on new commits. Configure here.
Note
Fix work log 'Show more' cursor, transition, and scroll behavior
Collapsible/CollapsibleTriggercomponents, showing 'Show N more' or 'Show less' based on overflow count.keepMounted) instead of being unmounted, fixing transition behavior.Macroscope summarized 66cb7cf.